Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move BandedMatrices+BlockArrays to BlockArray extension #195

Merged
merged 17 commits into from
May 18, 2024

Conversation

dlfivefifty
Copy link
Member

No description provided.

Copy link

codecov bot commented Apr 15, 2024

Codecov Report

Attention: Patch coverage is 95.45455% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 89.53%. Comparing base (1425899) to head (33fe6b9).
Report is 1 commits behind head on release-0.13.

Files Patch % Lines
src/BandedBlockBandedMatrix.jl 88.88% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release-0.13     #195      +/-   ##
================================================
+ Coverage         88.37%   89.53%   +1.15%     
================================================
  Files                11       11              
  Lines              1204     1108      -96     
================================================
- Hits               1064      992      -72     
+ Misses              140      116      -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jishnub
Copy link
Member

jishnub commented Apr 29, 2024

Let's create a release-0.13 branch, as this would simplify collecting all the changes.

Edit: I've created one now. Would you mind changing the target branch for this PR?

@dlfivefifty dlfivefifty changed the base branch from master to release-0.13 April 29, 2024 07:12
@dlfivefifty
Copy link
Member Author

Done. But what else is going in release-0.13?

@jishnub
Copy link
Member

jishnub commented Apr 29, 2024

There are some other minor changes that might be useful, like not exporting Zeros/Ones/Fill/Block and asking users to import these from the corresponding packages.

@dlfivefifty
Copy link
Member Author

What's wrong with exporting them?

@jishnub
Copy link
Member

jishnub commented Apr 29, 2024

It just makes the package more prone to breakages, as breaking changes upstream get translated to this package as well, which shouldn't be the case usually. Typically, one may bump the minor or patch version of a package to incorporate a breaking version of a dependency.

dlfivefifty and others added 4 commits April 29, 2024 21:39
Bumps [julia-actions/cache](https://github.com/julia-actions/cache) from 1 to 2.
- [Release notes](https://github.com/julia-actions/cache/releases)
- [Commits](julia-actions/cache@v1...v2)

---
updated-dependencies:
- dependency-name: julia-actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dlfivefifty
Copy link
Member Author

ok I see your point

@dlfivefifty dlfivefifty merged commit 0a403e7 into release-0.13 May 18, 2024
13 checks passed
@dlfivefifty dlfivefifty deleted the dl/moveoutbanded branch May 18, 2024 07:51
dlfivefifty added a commit that referenced this pull request May 18, 2024
* Don't export names not owned by the package (#203)

* Move BandedMatrices+BlockArrays to BlockArray extension (#195)

* Move some banded matrix stuff to BlockArrays extension

* Update interfaceimpl.jl

* Move over more code

* tests pass

* Update Project.toml

* v0.13

* Update BlockBandedMatrices.jl

* Update BlockBandedMatrices.jl

* updates

* Bump julia-actions/cache from 1 to 2 (#204)

Bumps [julia-actions/cache](https://github.com/julia-actions/cache) from 1 to 2.
- [Release notes](https://github.com/julia-actions/cache/releases)
- [Commits](julia-actions/cache@v1...v2)

---
updated-dependencies:
- dependency-name: julia-actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* PseudoBlockMatrix -> BlockedMatrix

* Update BlockSkylineMatrix.jl

* Update BandedBlockBandedMatrix.jl

* Move out banded code

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Jishnu Bhattacharya <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants